Skip to content

Example workflow fixes#150

Open
kolayne wants to merge 1 commit into
multimediallc:mainfrom
kolayne:patch-1
Open

Example workflow fixes#150
kolayne wants to merge 1 commit into
multimediallc:mainfrom
kolayne:patch-1

Conversation

@kolayne

@kolayne kolayne commented Jun 19, 2026

Copy link
Copy Markdown

Summary / Background

  • Remove the unnecessary issues: write permission;

  • Change actions/checkout version to latest: v4 is outdated, and codeowners-plus doesn't depend on its version-specific features.

Permission tested on: kolayne/test#3

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates the GitHub Actions configuration example in the README.md by removing the issues: write permission and changing the checkout action version from @v4 to @latest. The review feedback correctly identifies that using @latest will cause the workflow to fail since the actions/checkout repository does not maintain a latest tag, and recommends reverting back to @v4 for stability and security.

Comment thread README.md Outdated
steps:
- name: 'Checkout Code Repository'
uses: actions/checkout@v4
uses: actions/checkout@latest

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

high

Using actions/checkout@latest will cause the workflow to fail because the actions/checkout repository does not publish or maintain a latest tag. Additionally, pinning to mutable tags is a security and stability risk. It is best practice to pin to a specific major version (such as v4) or a specific commit SHA to ensure reproducible and secure builds.

Suggested change
uses: actions/checkout@latest
uses: actions/checkout@v4

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:/

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updated to v7, tested at kolayne/test#3

- Remove the unnecessary `issues: write` permission;

- Change `actions/checkout` version to `latest`:
  v4 is outdated, and codeowners-plus doesn't
  depend on its version-specific features.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant